-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Fix typos newly found by codespell #11735
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
picnixz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me (still you'll need to wait for Adam's approval).
90da3e4 to
8aed5e5
Compare
8aed5e5 to
cdcdfda
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of the .po files can be excluded from the spelling checker, they're automatically updated from the source.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's what I thought, but I had been told otherwise in #11735 (comment).
Any way, these updates are in a separate commit, they would be easy to discard.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we change a localized string, I think what is autogenerated is an empty translation, hence we probably need to manually update translations in this case but I may be wrong.
sphinx/writers/text.py
Outdated
| """ | ||
| if cell.row is None or cell.col is None: | ||
| msg = 'Cell co-ordinates have not been set' | ||
| msg = 'Cell coordinates have not been set' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
co-ordinates is fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both co-ordinate and coordinate are fine, especially in British English according to dictionaries. However, the Google Ngram Viewer shows coordinate in wider use, including in the British English corpus strangely enough.
It's up to you, but what's wrong with following common use when targetting a large and international audience?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even though co-ordinate is fine, I personally prefer one without "-". Also it removes one character from the code base so it's technically an optimization!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about a compromise and use "coördinates" like the New Yorker! 🙃
https://www.newyorker.com/culture/culture-desk/the-curse-of-the-diaeresis
Subject: fix typos
Feature or Bugfix